-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove core imports from dbt.artifacts component #9694
Conversation
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9694 +/- ##
=======================================
Coverage 88.08% 88.08%
=======================================
Files 178 180 +2
Lines 22459 22457 -2
=======================================
Hits 19782 19782
+ Misses 2677 2675 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice cleanup.
resolves ##9926
Problem
Non dbt.artifacts imports from dbt.* in dbt/artifacts represent circular dependencies if dbt/artifacts were to be pulled out as a standalone package from dbt-core.
Solution
This PR is best to review commit-by-commit!
Remove as many dbt.* core imports as possible in this PR. This has included:
Outstanding dependencies:
These are going to be a higher lift and I'd like to tackle them in a follow-on to prevent this PR from getting too large in scope.
Checklist